-
Notifications
You must be signed in to change notification settings - Fork 0
Analyze mobile header issues #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: lucaswan348 <lucaswan348@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: CSS Variable Scope Issue
The CSS variable --header-h is used for the navbar height and hero section padding-top, but is only defined within the mobile media query. This causes the variable to be undefined on desktop and larger screens, resulting in incorrect layout for these elements.
styles.css#L42-L181
Lines 42 to 181 in 3f8d65f
| z-index: 1000; | |
| height: var(--header-h); /* use the shared header height */ | |
| padding: calc(env(safe-area-inset-top) + 0.5rem) 0 0.5rem; /* safe-area + some breathing room */ | |
| border-bottom: 1px solid #333; | |
| box-shadow: 0 2px 20px rgba(0, 0, 0, 0.3); | |
| } | |
| .nav-container { | |
| max-width: 1200px; | |
| margin: 0 auto; | |
| padding: 0 20px; | |
| display: flex; | |
| justify-content: space-between; | |
| align-items: center; | |
| gap: 2rem; | |
| } | |
| .nav-logo { | |
| display: flex; | |
| flex-direction: column; | |
| align-items: flex-start; | |
| gap: 0.2rem; | |
| } | |
| .logo-text { | |
| font-size: 2rem; | |
| font-weight: 700; | |
| letter-spacing: -0.5px; | |
| } | |
| .logo-text .c-letter { | |
| color: #FF8C00; /* Orange */ | |
| text-shadow: 0 0 10px rgba(255, 140, 0, 0.3); | |
| } | |
| .logo-text .n-letter { | |
| color: #FFB6C1; /* Pastel Pink */ | |
| text-shadow: 0 0 10px rgba(255, 182, 193, 0.3); | |
| } | |
| .logo-subtitle { | |
| font-size: 0.85rem; | |
| color: #aaa; | |
| font-weight: 400; | |
| letter-spacing: 0.5px; | |
| text-transform: uppercase; | |
| } | |
| .nav-menu { | |
| display: flex; | |
| list-style: none; | |
| gap: 2.5rem; | |
| align-items: center; | |
| } | |
| .nav-menu a { | |
| color: #ffffff; | |
| text-decoration: none; | |
| font-weight: 500; | |
| font-size: 0.95rem; | |
| transition: all 0.3s ease; | |
| padding: 0.5rem 1rem; | |
| border-radius: 6px; | |
| position: relative; | |
| } | |
| .nav-menu a:hover { | |
| color: #ffffff; | |
| background: rgba(255, 255, 255, 0.1); | |
| } | |
| .nav-menu a::after { | |
| content: ''; | |
| position: absolute; | |
| bottom: 0; | |
| left: 50%; | |
| width: 0; | |
| height: 2px; | |
| background: linear-gradient(90deg, #FF8C00, #FFB6C1); | |
| transition: all 0.3s ease; | |
| transform: translateX(-50%); | |
| } | |
| .nav-menu a:hover::after { | |
| width: 80%; | |
| } | |
| .nav-menu a.active { | |
| color: #FF8C00; | |
| background: rgba(255, 140, 0, 0.1); | |
| } | |
| .nav-menu a.active::after { | |
| width: 80%; | |
| background: #FF8C00; | |
| } | |
| .hamburger { | |
| display: none; | |
| flex-direction: column; | |
| cursor: pointer; | |
| gap: 4px; | |
| padding: 0.5rem; | |
| border-radius: 6px; | |
| transition: all 0.3s ease; | |
| border: none; | |
| background: transparent; | |
| } | |
| .hamburger:hover { | |
| background: rgba(255, 255, 255, 0.1); | |
| } | |
| .hamburger span { | |
| width: 25px; | |
| height: 3px; | |
| background: #ffffff; | |
| border-radius: 2px; | |
| transition: all 0.3s ease; | |
| } | |
| /* Hamburger animation */ | |
| .hamburger.active span:nth-child(1) { | |
| transform: rotate(45deg) translate(5px, 5px); | |
| } | |
| .hamburger.active span:nth-child(2) { | |
| opacity: 0; | |
| } | |
| .hamburger.active span:nth-child(3) { | |
| transform: rotate(-45deg) translate(7px, -6px); | |
| } | |
| /* Hero Section */ | |
| .hero { | |
| min-height: 100vh; | |
| display: flex; | |
| align-items: center; | |
| padding-top: var(--header-h); |
Bug: Mobile Navigation Menu Disappears
The hamburger menu is not visible on mobile devices. The display: flex; property was removed from the mobile media query, while the base .hamburger style retains display: none;. This prevents the menu from appearing, making mobile navigation inaccessible.
styles.css#L858-L862
Lines 858 to 862 in 3f8d65f
| } | |
| .hamburger { | |
| margin-left: auto; | |
| } | |
BugBot free trial expires on August 4, 2025
Learn more in the Cursor dashboard.
Was this report helpful? Give feedback by reacting with 👍 or 👎
Co-authored-by: lucaswan348 <lucaswan348@gmail.com>
Implement mobile header fixes to resolve layout, accessibility, and responsiveness issues.
Open Background Agent:
Web · Cursor
Learn more about Background Agents